chore: upgrade @oclif/core to v4 and eslint-config to v5#241
Merged
Conversation
…fig to v5 - Bump @oclif/core from ^2 to ^4 - Replace ux.table() (removed in v4) with a custom table utility that replicates the same output format (src/utils/table.js) - Patch Command.prototype.parse in jest setup to provide a mock config.runHook, required by @oclif/core v4 when commands are instantiated directly in tests - Upgrade @adobe/eslint-config-aio-lib-config from v4 to v5 (ESLint 9 flat config) - Replace .eslintrc.json with eslint.config.js using new flat config format - Upgrade eslint from ^8 to ^9 and eslint-plugin-jest to ^29 - Add neostandard as peer dependency; remove eslint-plugin-node, eslint-config-standard, eslint-plugin-import, eslint-plugin-n, eslint-plugin-promise (all superseded by neostandard) - Remove stale eslint-disable directives obsoleted by the new config Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Agent-Logs-Url: https://github.com/adobe/aio-cli-plugin-console/sessions/12aa3031-264b-4dc9-8dfb-910621dbffd2 Co-authored-by: shazron <36107+shazron@users.noreply.github.com>
Fixed in f28d7d8 — updated copyright year to 2026 in both new files. |
Member
Author
|
@copilot the PR description was replaced, it should not have been. Recreate it (from commit logs etc) since we can't restore a description. |
Restored the original PR description from the commit logs and the previous description content. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #240
Summary
@oclif/corefrom^2to^4(resolves chore: update @oclif/core to latest version #240)@adobe/eslint-config-aio-lib-configfrom v4 to v5 (ESLint 9 flat config).eslintrc.jsontoeslint.config.jsKey changes
@oclif/corev4 migration:ux.table()was removed in v4 — replaced withsrc/utils/table.js, a custom renderer that produces identical output (verified against all test fixtures)Command.prototype.parse()now requiresthis.config.runHook— added a mock intest/jest.setup.jsfor tests that instantiate commands directlyeslint-plugin-node'sno-missing-requirerule can't resolve packages with only anexportsfield (nomain) — resolved by migrating to neostandard which useseslint-plugin-nESLint v9 / flat config migration:
eslint-config-standard,eslint-plugin-node,eslint-plugin-n,eslint-plugin-import,eslint-plugin-promisewithneostandardeslint-plugin-jestto^29for ESLint 9 flat config supporteslint-disabledirectives that were suppressing rules no longer presentTest plan